Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix cheatsheet_app.h correct types for oc_on_resize function #103

Closed
wants to merge 1 commit into from

Conversation

RokKos
Copy link
Contributor

@RokKos RokKos commented Feb 21, 2025

I was going through samples and the cheat sheet for the app and noticed that there are wrong types in the cheat sheet.

@martinfouilleul
Copy link
Collaborator

martinfouilleul commented Mar 2, 2025

thanks for pointing this out! yeah the cheatsheets are out of date in a few other places too, and tbf I don't know if we want to keep them at all, now that we have the API docs at https://docs.orca-app.dev...
Maybe I'll just remove them altogether

@RokKos
Copy link
Contributor Author

RokKos commented Mar 3, 2025

In my novice opinion with Orca (and biased opinion coming from raylib) I really like the cheat sheet and it's easier to get quickly off the ground. The API docs are really nice, checked them when I wanted to see the implementation.

But I am also aware from your blog post that it's hard to maintain this cheat sheet.

So I can see your point of removing them altogether

@martinfouilleul
Copy link
Collaborator

Yeah I agree that cheatsheets are more light-weight than the docs and can be useful to try things out quickly. I think we could have a python script that generates cheatsheets from our api.json declarations now that it's a bit stabilized, it just needs a little work.
I'll create an issue for that and I (or someone else) will get to it eventually, but in the meantime I would prefer not to have outdated/misleading cheatsheets, so I'll temporarily remove them.

@martinfouilleul
Copy link
Collaborator

follow-up here: #106

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants